Hotfix - Proyectos - Inclusión de ficheros javascript en view.detail.php #529
Conversation
|
Actions executed at: 2025-01-21 09:16:59. |
There was a problem hiding this comment.
Se ha verificado el comportamiento: Se incluyen correctamente los ficheros .js aunque el detailviewdefs.php no los incluya.
Cuestión: ¿Porqué no se eliminan los includes del detailviewdefs? Estos .js se incluirán siempre al cargar la vista de detalle, y versionados. Esto hará que se carguen dos veces como dos recursos distintos (uno sin versionar y otro versionado). No veo mucho problema en eso, pero yo los quitaría
De acuerdo con la propuesta, se han eliminado la inclusión redundante de los ficheros en el detailviewdef a favor de la inclusión versionada en view.detail.php |
No tengo claro cómo hemos de reflejar esta situación. Si se trata del fichero modules/Project/metadata/detailviewdefs.php este ya tiene un bloque STIC Custom, pero nosotros estamos haciendo cambios dentro de nuestro propio bloque de código, y entiendo que aquí no debemos añadir nada más ¿no?. Lo que sí he hecho es añadirlo al proyecto de modificaciones del core como Internal STIC |
Description
Para evitar el problema descrito en el issue se ha optado por hacer la inclusión de los ficheros Javascript usados en la vista de detalle de Proyectos directamente en el controlador de la vista
custom/modules/Project/views/view.detail.php.Además del fichero faltante detectado en el issue (modules/Project/js/custom_project.js), se ha añadido el fichero modules/Project/Project.js, por igualar la estratégia, al menos en este fichero.
Cómo probarlo